-
Notifications
You must be signed in to change notification settings - Fork 29.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
report: fix build warning in node_report.cc #26265
Conversation
@richardlau sadly an error occured when I tried to trigger a build :( |
CI will need to wait until the lockdown ends. |
CI looks quiescent right now, so here you go: https://ci.nodejs.org/job/node-test-pull-request/20971/ Not sure if the GitHub widget will still report results or if someone with access will have to let you know if it passed or not. |
The status is being posted, so the widget is showing results. FTR I'm in no particular rush to land this. |
@richardlau The Windows failure is a pervasive infra-related thing. If you're in no rush, once it's fixed, we can do a quick Resume Build. |
I think @refack fixed stuff. 🎉 Here's a Resume Build CI: https://ci.nodejs.org/job/node-test-pull-request/20972/ |
1fc7bb9
to
ec6420c
Compare
Rebased based on irc chat suggestions that the Windows issue is #25593 and that recent commits to master may have mitigated it. |
CI: https://ci.nodejs.org/job/node-test-pull-request/20974/ I don't think it's been fixed yet, but I'll be happy to be wrong.... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Resume CI: https://ci.nodejs.org/job/node-test-pull-request/21031/ (✔️) |
Fixes `maybe-uninitialized` build warning in `src/node_report.cc`. PR-URL: nodejs#26265 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
ec6420c
to
4a10ce6
Compare
Landed in 4a10ce6. |
Fixes `maybe-uninitialized` build warning in `src/node_report.cc`. PR-URL: #26265 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Fixes the following build warning:
Checklist
make -j4 test
(UNIX), orvcbuild test
(Windows) passes